Skip to content

docs(rfc): record the measured 1 MiB budget closure - #8340

Open
LIHUA919 wants to merge 1 commit into
loopx-project:mainfrom
LIHUA919:codex/rfc-matched-1m-closure
Open

LIHUA919 wants to merge 1 commit into
loopx-project:mainfrom
LIHUA919:codex/rfc-matched-1m-closure

Conversation

@LIHUA919

@LIHUA919 LIHUA919 commented Nov 8, 2026

Copy link
Copy Markdown
Contributor

Refs #4224.

What changes

The RFC's 2026-09-24 checkpoint (both languages) still said the 1 MiB receipt/scan budgets were failed and that #4931's merge was "read-proof optimization, not proof". Since then the formal matched-1m re-measure ran at head a9485073f (containing the merged read-proof reuse) on a measurement entrypoint repaired by #8256 (merged 2026-11-08). This PR records the closure as a dated checkpoint in shared-goal-authority-state-provider-v0.md and shared-goal-authority-state-provider-v0.zh-CN.md:

Validation

  • scripts/check_rfc_bilingual.py --base origin/main --staged passed (both files change together).
  • Exact-path loopx check boundary scan clean on both files; git diff --check clean. Docs-only change; no code, tests or thresholds touched.

The 2026-09-24 checkpoint still said the 1 MiB receipt/scan budgets were
failed and that loopx-project#4931 was only an optimization without proof. The formal
matched-1m re-measure has since run at head a948507 (containing the
merged read-proof reuse), on a measurement entrypoint repaired by loopx-project#8256:
receipt p95 26.44/27.65 ms and scan-100 p95 60.85/69.62 ms at 10k/100k,
both inside the 50/250 ms budgets with no depth degradation, ledger 13
passed / 0 failed / 11 missing.

Record that closure as a dated checkpoint in both languages, with the
boundary stated equally clearly: the soak failed at 9d22.85h and the
recovery, consumer-lag and OS/runtime matrices remain missing, so this
closes budget rows, not D2.

Refs loopx-project#4224

Signed-off-by: Lihua <1017343802@qq.com>

@loopx-agent loopx-agent left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewer: model_agent | gpt-6.1-sol | OpenAI | runtime_reported | reasoning_effort=max

动机 / Motivation

维护 SQLite 资格记录的人,需要知道哪些历史预算已经重新测量,哪些长期验收仍然缺证据。原检查点只保留九月的 1 MiB receipt/scan 失败;新增带日期的段落对照后来公开的正式复测,同时保留原失败、失败 soak 及 D2 未完成边界。两份语言镜像准确转录公开记录中的四个 p95、50/250 ms 阈值、13/0/11 账本及未完成项,未修改阈值或宣布 D2 通过。本 PR 只维护文档,不重新运行容量或自然时间实验,不修改存储、默认 provider、预算、测试和晋升权限。

改动思路 / Approach

保留旧检查点,再追加一条有日期、有来源、有限范围的复测说明,比覆盖旧失败更便于后续资格判断。这里的有效结果是文档与已公开记录一致,不是本评审重新签发性能或 D2 证书。没有新增配置或用户操作,也没有让后续独立改进等待整个 D2 矩阵。

具体改动 / Changes

关键内容讲解

准确 head 64c4798745e96c3efad812b453d46dfc7cb65f85,两份既有 RFC 镜像共 +20/-0。英文第 3440 行附近和中文第 2671 行附近追加同一段:10k/100k 的 receipt p95 为 26.44/27.65 ms,scan-100 为 60.85/69.62 ms;对应预算 50/250 ms、账本 13 passed / 0 failed / 11 missing。两种语言均保留失败 soak 9d22.85h、恢复、consumer-lag 与 OS/runtime 缺口,明确这些行通过不等于 D2。

逐项核对的是作者的正式复测记录和失败 soak 记录。公开记录声明 Node 22.22.3 / SQLite 3.51.3、macOS arm64、WAL/FULL;数值、历史对照与剩余缺口均相符。

规范依据:docs/architecture/rfcs/shared-goal-authority-state-provider-v0.md,固定修订 94366314f0ba281b97b6ff2fc90b929320b163af,7.2 implemented 仅指保持预算、原失败、工作负载轴及资格分层的文档约束。真实自然时间及完整 D2 资格仍未由这个 PR 交付。

来源边界也单独核对了:a9485073f 对应 a9485073f219a416689f62824c5117269289e596;#8256 于 2026-11-08 05:12:15Z 合并,其 merge 不是该旧源码的祖先。因此应区分记录中的存储测量基线和另行修复的 measurement runner,不能理解为这个旧 commit 自身已经包含后来合并的 runner 修复。本审未拿两套源码或旧报告拼接成一次新的正式测量。

对主干的风险 / Risks

未发现阻断问题。 主要风险是把局部预算关闭误读成 provider 晋升、十天资格或全平台通过;新增双语段落明确保留反例和 missing,原九月失败段落、阈值与实现均未改。没有必要为这项记录同步重新启动实验或改运行时。

标准 premerge 的 4 direct/14 selected、双语配对差分及公开边界检查 PASS;手工核对两种语言的数字、单位、日期和否定边界一致。没有独立重跑 10k/100k、soak、恢复/consumer-lag 或 OS/runtime 矩阵,也没有查询远端 CI;这些未被记作本审通过。

我的整体评价 / Verdict

APPROVE。 这是准确保存来源和剩余义务的完整文档增量。long_horizon 为 preserved:未改变续跑、状态或资格门;user_experience 为 improved,仅指维护者在原 RFC 即可同时看到新测量与旧失败,未宣称产品运行收益。视觉检查为 not_applicable:两份 Markdown 检查点不改渲染代码或信息层级;未验证范围是文档站实际渲染及上述性能/耐久矩阵。无新增共享词汇、默认值或兼容分支,保留原章节即可;没有合并授权。

English verdict: APPROVE — 64c4798; the bilingual checkpoint matches the cited measurements and preserves failed/missing D2 evidence. This is documentation validation, not a new performance qualification.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants